Skip to content

fix organization pagination conflict#1497

Merged
Nkwenti-Severian-Ndongtsop merged 10 commits intomainfrom
1493-fix-organization-pagination-conflict
Apr 14, 2026
Merged

fix organization pagination conflict#1497
Nkwenti-Severian-Ndongtsop merged 10 commits intomainfrom
1493-fix-organization-pagination-conflict

Conversation

@Nkwenti-Severian-Ndongtsop
Copy link
Copy Markdown
Collaborator

What this PR does / why we need it:
: Fixes organization pagination conflict when importing realms with more than 10 organizations.

Which issue this PR fixes fixes #1493

Screenshots / logs (if relevant):


Screenshot from 2026-04-08 09-10-03

Special notes for your reviewer:

  • Pagination implementation follows the same pattern as ClientRepository.findAll()
  • Uses page size of 100 for consistency with other repositories
  • Replaces deprecated .getAll() with paginated .list(firstResult, maxResults)
  • No breaking changes - maintains existing API while fixing the underlying pagination issue

PR Readiness Checklist:

  • Code follows project style guidelines
  • Added tests that prove the fix is effective
  • No checkstyle violations

Complete these before marking the PR as ready to review:

  • tests pass locally (mvn test or relevant subset)
  • added/updated tests for the changes (if applicable)
  • documentation has been updated (if applicable)
  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 8, 2026

@Nkwenti-Severian-Ndongtsop
Copy link
Copy Markdown
Collaborator Author

Nkwenti-Severian-Ndongtsop commented Apr 8, 2026

@Nkwenti-Severian-Ndongtsop needs to provide a detailed documentation on Org features and compatibility with Keycloak version and client library version as well

Copy link
Copy Markdown
Collaborator

@mbiti2 mbiti2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Nkwenti-Severian-Ndongtsop it works for me

@Nkwenti-Severian-Ndongtsop Nkwenti-Severian-Ndongtsop merged commit 210367d into main Apr 14, 2026
15 checks passed
@Nkwenti-Severian-Ndongtsop Nkwenti-Severian-Ndongtsop deleted the 1493-fix-organization-pagination-conflict branch April 14, 2026 09:11
@github-project-automation github-project-automation bot moved this from Review to Done in os-competence-center-board Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Importing organizations fails if there are >10 organizations being updated

3 participants